Add Codex parity tooling#154
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Storybook Component Screenshot PreviewOutcome: success Screenshots: primitive component stories captured on desktop and mobile. This lane is separate from full-route Playwright screenshots and focuses on design-system component regressions. |
Storybook Image DiffOutcome: success Changed Storybook baselines: none in this PR. This check compares design-system component screenshots against committed baselines. Inline images show only added or modified Storybook baseline PNGs. |
Greptile SummaryThis PR adds Codex parity tooling and protected-main workflow guards. The main changes are:
Confidence Score: 4/5The workflow guard additions are mostly isolated, but one Codex skill wrapper references a missing source file and should be fixed before relying on the new VRCDN onboarding path. The changed files are primarily configuration, package scripts, and documentation wrappers, with a narrow tooling failure confirmed around the VRCDN skill reference. .codex/skills/vrcdn/SKILL.md
What T-Rex did
|
Playwright Hosted Data-FlowOutcome: success This optional check runs the mutation-backed profile flow against a configured hosted dev/staging target with isolated E2E test data. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f6a1b37eac
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Playwright Public Screenshot PreviewOutcome: success Screenshots: all public route checks passed on desktop and mobile. Full screenshot set is available in the artifact. Pixel diff baselines are handled by the separate Playwright Image Diff check. |
Playwright Image DiffOutcome: success Changed screenshot baselines: none in this PR. This check compares public route screenshots against committed baselines. Inline images show only added or modified baseline PNGs. |
Playwright Data-Flow PreviewOutcome: success Captured flow:
Artifacts include screenshots, traces, and recorded video for the flow run. |
Vercel Preview DeploymentPreview: https://vr-dex-bjixrd2wn-basicbit.vercel.app |
There was a problem hiding this comment.
Pull request overview
Adds repo-local Codex parity tooling and workflow guardrails for VRDex by introducing a Codex MCP config + thin skill wrappers, documenting Codex workflow notes, and enforcing a “protected main mirror” policy via npm/pnpm script guards.
Changes:
- Add
scripts/guard-main-worktree.mjsand wire it into root + workspacepre*scripts to block local execution onmain(with explicit bypasses). - Add Codex documentation (
docs/agentic/codex.md) and link it from the agentic docs index. - Add repo-local Codex scaffolding:
.codex/config.toml,.codex/skills/*wrappers, and.codex/state/gitignore.
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/guard-main-worktree.mjs | New Node script that blocks running local workflows on main unless explicitly bypassed. |
| package.json | Adds guard script and many pre* hooks so common root workflows invoke the guard. |
| docs/agentic/README.md | Adds the Codex notes doc to the agentic docs index. |
| docs/agentic/codex.md | Documents Codex workflow mapping, MCP config location, and protected-worktree policy. |
| apps/web/package.json | Adds pre* guards for local web app scripts (dev/build/test/typecheck/etc.). |
| apps/docs/package.json | Adds pre* guards for docs app scripts. |
| apps/discord-gateway/package.json | Adds pre* guards for discord-gateway scripts. |
| AGENTS.local.md.example | Extends the local agents example with a “worktree role” note. |
| .gitignore | Ignores .codex/state/ alongside existing OpenCode state. |
| .codex/skills/vrdex-onboarding/SKILL.md | New thin Codex wrapper pointing to the OpenCode onboarding skill. |
| .codex/skills/vrcdn/SKILL.md | New thin Codex wrapper pointing to the OpenCode VRCDN skill with safety reminders. |
| .codex/config.toml | New project-scoped Codex MCP server config mirroring the OpenCode server list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
Verification